Merged
Conversation
JulienLavocat
changed the base branch from
master
to
tien/feat/svelte-integration
cloutiertyler
changed the base branch from
tien/feat/svelte-integration
to
master
cloutiertyler
changed the base branch from
master
to
tien/feat/svelte-integration
clockwork-labs-bot
changed the base branch from
tien/feat/svelte-integration
to
master
# Description of Changes The PR introduces refinements for the Angular integration: - Convert chat app to a basic template app, and fixes issues with template integrations - Adds Quickstart docs for Angular - Angular sdk improvements incl. connection state handling The Angular integration requires merging the latest master afterwards, which wasn't included to simplify review. # Screenshots - Angular template <img width="1198" height="849" alt="image" src="https://github.com/user-attachments/assets/67f9dd57-1a4b-4aa2-8a3d-2bf0e2f61b80" /> - Angular Quickstart docs <img width="1458" height="897" alt="image" src="https://github.com/user-attachments/assets/04ed0376-cb2d-4f4c-8f6b-adfc77e38d3e" /> <!-- Please describe your change, mention any related tickets, and so on here. --> # API and ABI breaking changes <!-- If this is an API or ABI breaking change, please apply the corresponding GitHub label. --> # Expected complexity level and risk 1 # Testing <!-- Describe any testing you've done, and any testing you'd like your reviewers to do, so that you're confident that all the changes work as expected! --> - [x] Works well from my testing
Resolve all conflicts by taking master's changes for non-Angular files. Update Angular inject-table.ts to use new query API (query.ts) instead of deleted filter.ts. Update Angular exports and build config accordingly.
- Re-export ParamsType from sdk/index.ts (lost during merge) - Add @angular/core as peer + dev dependency for type resolution - Remove unused UseTableRowType in inject-table.ts - Add explicit type for onCleanup parameter in Angular effects
- Regenerate module_bindings with v2.0.0 codegen (copied from react-ts, same module schema) - Update module source to use current schema API format - Fix withModuleName -> withDatabaseName in app.config.ts - Set noPropertyAccessFromIndexSignature: false in tsconfig.json (avoids TS4111 on row property access from index signatures) - Update spacetimedb dependency to workspace:*
cloutiertyler
approved these changes
Feb 16, 2026
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of Changes
API and ABI breaking changes
None.
Expected complexity level and risk
2
Testing
Ran the template